Skip to content

Conversation

@bassiounix
Copy link
Contributor

No description provided.

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@bassiounix bassiounix marked this pull request as ready for review August 3, 2025 18:14
@llvmbot llvmbot added the libc label Aug 3, 2025
@llvmbot
Copy link
Member

llvmbot commented Aug 3, 2025

@llvm/pr-subscribers-libc

Author: Muhammad Bassiouni (bassiounix)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/151879.diff

1 Files Affected:

  • (modified) libc/src/__support/math/cbrt.h (-1)
diff --git a/libc/src/__support/math/cbrt.h b/libc/src/__support/math/cbrt.h
index 2b9a73c823b14..360e68aafb75d 100644
--- a/libc/src/__support/math/cbrt.h
+++ b/libc/src/__support/math/cbrt.h
@@ -146,7 +146,6 @@ LIBC_INLINE static constexpr double get_error(const DoubleDouble &x_3,
 
 LIBC_INLINE static constexpr double cbrt(double x) {
   using DoubleDouble = fputil::DoubleDouble;
-  using Float128 = fputil::DyadicFloat<128>;
   using namespace cbrt_internal;
   using FPBits = fputil::FPBits<double>;
 

@bassiounix bassiounix merged commit d7f00a9 into main Aug 3, 2025
19 checks passed
Copy link
Contributor Author

Merge activity

@bassiounix bassiounix deleted the users/bassiounix/spr/08-03-_libc_fix_build_errors_caused_by_cbrt branch August 3, 2025 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants